Indentation cleanups in linux driver code.
Signed-off-by: Keir Fraser <keir@xensource.com>

Merged?

A more comprehensive fix for mapping shared-ring grant
references in back-end drivers.

Signed-off-by: Keir Fraser <keir@xensource.com>

This patch adds a check in xend that prevents a user from trying to
destroy Dom0.  Currently, xm does not return an error for this case.
Xend tries to destroy dom0 and leaves the system in an unstable
state.  Subsequent attempts to create a domain result in the following
error message:

  Error: Error creating domain: (106, 'Transport endpoint is already \
  connected')

This will fix bugzilla bug #243

Signed-off-by: Dan Smith <danms@us.ibm.com>

VMX guest memory calculation in python code is corrupted by changeset
6984, this patch fixes it.

Also print vmx builder parameters before calling vmx builer, just like
what's done before calling linux builder.

With this patch, qemu dm using SDL can work, however qemu dm using VNC
still has bug, we will fix it soon.

Signed-off-by: Xin Li <xin.b.li@intel.com>

this patch fixes the bug that when "cpu" is not set in config file,
control panel complains "Error creating domain - int argument required".
Signed-off-by: Xin Li <xin.b.li@intel.com>

This fixes the "make check" feature of xenstore to properly compile with
x86-64. When compiling you would get the following error since on x86-64
it was passing a long , when you are expecting an int.

Signed-off-by: Jerone Young <jyoung5@us.ibm.com>

Trivial spelling mistake fix.
... but mainly to check that pushes to the repository are working

Signed-off-by: James Bulpin <james@xensource.com>

This patch fixes a bug where raise_softirq(SCHEDULE_SOFTIRQ) is called
upon a hlt instruction from a VMX guest, causing repeated VMExits when
the guest is idle. At the same time, it disables the monitor/mwait
feature as it's not feasible to implement for vcpu.

Signed-off-by: Xiaofeng Ling <xiaofeng.ling@intel.com>
Signed-off-by: Asit Mallick <asit.k.mallick@intel.com>
Signed-off-by: Jun Nakajima <jun.nakajima@intel.com>

Fix mapping of shared ring pages into backend drivers.
This should fix observed problems when creating a domain,
causing Xen to print "Could not find PTE entry for
address ...".

Signed-off-by: Keir Fraser <keir@xensource.com>

Merge.

Split the configuration file parsing and xc dominfo parsing aspects of
XendDomainInfo out from the rest of the code, creating an intermediate
dictionary of common format.  This means that the rest of XendDomainInfo can
work whether creating a domain for the first time, or whether xend is
restarted but the domains still exist.  This means that xend is now robust in
the face of restarts.  The phase of validation of configuration details is
now more explicit, and stronger too.

Change the handling of memory sizes in XendDomainInfo's interface to use KiB
in setMemoryTarget and getMemoryTarget.  This gives reasonable granularity
whilst ensuring that we can handle values up to 2TiB without overflowing a
signed 32 bit value.  It is not clear that the xend code, especially the C /
Python interface, is either 64-bit clean or unsigned 32-bit clean, so this is
the safest choice for now.  The behaviour of Python's shift operator will
change in Python 2.4, so it is best to address this problem once we have moved
to that version.

Determine the backend flags on-the-fly, in getBackendFlags, rather than
computing them in advance in configureBackends and storing the value.  Change
addControllerClass so that backend flag information is in this table too,
rather than hard-coded elsewhere.

Improve the error reporting for name clashes.

Remove XendDomainInfo's dependence upon DBMap, and use xstransact directly
instead.  This changes the interface from XendDomain to XendDomainInfo, as
paths rather than DBMaps are passed in.

Remove the XendDomainInfo.recreate and restore flags.  Since the device and
domain handling is now stateless inside xend, much less work is necessary for
recreate and restore procedures, so we can do without these flags.

Remove XendDomainInfo's unused dependency upon SrvDaemon, and its unnecessary
dependence upon PrettyPrint.  Remove the unused show method.

Decouple image.py from XendDomainInfo.bootloader by passing a bootloading flag
into initDomain instead.  Decouple it from XendDomainInfo.config by passing
the semiparsed device configuration into create().  Move configuration in
VmxImageHandler so that rather than being scattered around the class it is in
or called from the configure method.  Device configuration is no longer
available anywhere else.

From Dan Smith <danms@us.ibm.com>:

      I could not find in the existing code the point at which
      a domain was added to the XendDomain list after restore.  Several
      attempts to restore would result in positive log messages, but the
      restored domain would not show up in "xm list".  This patch includes a
      call to _add_domain(), which results in restore working for me.

Signed-off-by: Ewan Mellor <ewan@xensource.com>

Disable debugging-level output.

Signed-off-by: Ewan Mellor <ewan@xensource.com>

Match change to the memory/target value, as it is now in KiB, not bytes.

Signed-off-by: Ewan Mellor <ewan@xensource.com>

Remove unused DEBUG variable, and -d flag.

Signed-off-by: Ewan Mellor <ewan@xensource.com>

Removed unused getLevel method, and don't use intermediate variable called
'log', as this name hides the global logger instance.

Signed-off-by: Ewan Mellor <ewan@xensource.com>

Added diagnostic messages to the RuntimeError exceptions when they occur inside
_read and _write.  Fix gather to not return the string 'None' when reading a
value of type string that does not exist.  The value None should be returned
instead.

Signed-off-by: Ewan Mellor <ewan@xensource.com>

Lots of minor changes to please pylint, including matching the number of
parameters between functions in the base classes and those that override them
in subclasses.  It's hard to see how this code worked at all.

Signed-off-by: Ewan Mellor <ewan@xensource.com>

On one of pyxc_domain_getinfo's error paths, free the block allocated
previously.  This would have caused a memory leak when attempting to get info
on a domain that does not exist.

Signed-off-by: Ewan Mellor <ewan@xensource.com>

Some cleanup in tpm-related files.

Signed-off-by: Stefan Berger <stefanb@us.ibm.com>

Removes redundant/unnecessary __vmread/__vmwrite.

Signed-off-by: Jun Nakajima <jun.nakajima@intel.com>
Signed-off-by: Eddie Dong <eddie.dong@intel.com>
Signed-off-by: Edwin Zhai <edwin.zhai@intel.com>

Parse vcpus for VMX guest.

Vcpus are passed to device model and vmx domain builder.

Signed-off-by: Yan Li <yanx.li@intel.com>
Signed-off-by: Xin Li <xin.b.li@intel.com>

Fix .hgignore and add missing vtpm patch file.

Remove unused variables and imports.

Signed-off-by: Ewan Mellor <ewan@xensource.com>

domain_setmaxmem takes an int, not a long, for the maxmem_kb parameter.  The
underlying xc_domain_setmaxmem already took an unsigned int, and
PyArg_ParseTupleAndKeywords call was only parsing an int, so there is no way
that longer values would get through here in any case.  Fixing the documentation
and the local variable is the best solution, until someone decides that we need
to support maxmem values greater than 2TiB.

Signed-off-by: Ewan Mellor <ewan@xensource.com>

Remove extraneous semicolon.

Signed-off-by: Ewan Mellor <ewan@xensource.com>

Move definition of xc, to avoid confusing pylint.

Signed-off-by: Ewan Mellor <ewan@xensource.com>

Remove unused import.

Signed-off-by: Ewan Mellor <ewan@xensource.com>

Mark unused variables as such.  Remove unused import.

Signed-off-by: Ewan Mellor <ewan@xensource.com>

Move definition of xc variable, to save confusing pylint.

Signed-off-by: Ewan Mellor <ewan@xensource.com>

Mark unused variable as such.

Signed-off-by: Ewan Mellor <ewan@xensource.com>

Added debugging message to exception on write.

Signed-off-by: Ewan Mellor <ewan@xensource.com>

The vmx guest loading is broken from 6925 because of the new
DevController code.

Signed-off-by: Xiaofeng Ling <xiaofeng.ling@intel.com>

Fix __save_flags() to not complain when smp_processor_id() is used
in a preemptible region (it is always safe).

Signed-off-by: Keir Fraser <keir@xensource.com>

merge?

Add support to build arch/ia64 xen kernels, also add pre/post link hooks in mkbuildtree.
Signed-off-by: Christian Limpach <Christian.Limpach@cl.cam.ac.uk>

Fix xenconsole when console page is >= 4GB.
Signed-off-by: Keir Fraser <keir@xensource.com>

Put_page on vmx_set_cr0 for multiple protected/real mode switch.

Signed-off-by: Yunhong Jiang <yunhong.jiang@intel.com>
Signed-off-by: Chengyuan Li <chengyuan.li@intel.com>

merge?

Fix vnc configuration issue for creating vmx guest.
Signed-off-by: Xiaofeng Ling <xiaofeng.ling@intel.com>
Signed-off-by: Christian Limpach <Christian.Limpach@cl.cam.ac.uk>

Split up docs. Signed-off-by: Robb Romans <3r@us.ibm.com>

Add 64 bit support to the VTPM Tools plus do some minor cleanups.

The VTPM manager and VTPMs fully support both 32 and 64 bit OSes. The
tpm_emulator (provided for debugging on TPM-less machines) does not
support 64-bit kernels by default though. See the README for details on
how to use it on 64-bit kernels.

(Vinnie Scarlata, Intel Corporation)

Signed-off-by: Joe Cihula <joe.cihula@intel.com>

Fix an operand size bug in the VMX MMIO decoder.

Signed-off-by: Chengyuan Li <chengyuan.li@intel.com>

Dynamically generate the local apic entries in ACPI MADT table.
The number of local apic entries is decided by the #vcpus passed
from the config file (eg./etc/xen/xmexample.vmx).

This feature is required by the SMP VMX domain.

Signed-off-by: Ke Yu <ke.yu@intel.com>
Signed-off-by: Xin Li <xin.b.li@intel.com>
Signed-off-by: Asit Mallick <asit.k.mallick@intel.com>

Add lomount to 'make uninstall' target. Remove miniterm,
as it's never installed on the Xen host.

Clean up 'make uninstall' target.

merge?

Fix kernel users of xs_write as well.
Signed-off-by: Christian Limpach <Christian.Limpach@cl.cam.ac.uk>

Better gdb server documentation.

Do not build PAE by default. Reverts accidental previous
checkin.

Fix xc_ptrace (live debug, not coredump debug) for 32-bit pae.
Clean up interfaces and add simple documentation for using
the domU debug interfaces.

Signed-off-by: Keir Fraser <keir@xensource.com>

Fire watches once immediately upon registration.
It matches the model of programming used to avoid races with watches,
and also makes re-establishing watches on daemon restart easier.
Signed-off-by: Rusty Russell <rusty@rustcorp.com.au>
Signed-off-by: Christian Limpach <Christian.Limpach@cl.cam.ac.uk>

Fix one more usage of xs_write.
Signed-off-by: Christian Limpach <Christian.Limpach@cl.cam.ac.uk>

Remove iflag argument to xs_write
xs_write with O_CREAT|O_EXCL causes problems over daemon restarts, since
it's not idempotent.
It turns out noone really needs the flags word at all, so get rid of it.
It's now as if everyone specified "O_CREAT".
Signed-off-by: Rusty Russell <rusty@rustcorp.com.au>
Signed-off-by: Christian Limpach <Christian.Limpach@cl.cam.ac.uk>

Make xs_mkdir an xs_rm idempotent.
When modifying libxenstore to transparently restart when the daemon dies,
it became apparent that life is simpler when all commands can simply be
restarted.  So this patch makes a slight semantic change to xs_rm and xs_mkdir:
xs_rm now succeeds if the file doesn't exist (as long as the parent exists),
and xs_mkdir succeeds if the directory already exists.
Noone should notice.
Signed-off-by: Rusty Russell <rusty@rustcorp.com.au>
Signed-off-by: Christian Limpach <Christian.Limpach@cl.cam.ac.uk>

Fix gdb build script.

merge?

merge?

Don't do device create on domain recreate.
Also don't add XendDomainInfo object to XendDomain dictionary from
XendDomainInfo object but do it in XendDomain.
Signed-off-by: Christian Limpach <Christian.Limpach@cl.cam.ac.uk>

Revert accidental commit.

Signed-off-by: Ewan Mellor <ewan@xensource.com>

Use DevController.allocateDeviceID rather than using a local variable.

Signed-off-by: Ewan Mellor <ewan@xensource.com>

Use DevController.allocateDeviceID rather than using a local variable.

Signed-off-by: Ewan Mellor <ewan@xensource.com>

Re-indent libxc to avoid hard tabs. Also, fix the PAE
domain builder to correctly write PTEs that map pages
above 4GB.

Signed-off-by: Keir Fraser <keir@xensource.com>

Remove last remaining "import controller" occurences.
Signed-off-by: Christian Limpach <Christian.Limpach@cl.cam.ac.uk>

merge?

Fix VMX domains not cleaning up properly, since some page refncts are
not zero. One reason for this is that in function shadow_set_l1e(),
before calling function shadow_update_min_max(), one should call
__shadow_get_sl2e() again, since the sl2 entry may be changed.

Signed-off-by: Xin Xiaohui <Xiaohui.xin@intel.com>
Signed-off-by: Li Chengyuan <Chengyuan.li@intel.com>

The domain structure maintains several shadow mode stats,
such as shadow page counts for l1 & l2, hl2 tables, snapshots,
etc.  These counts are not decremented properly when we
free shadow pages.  The following patch fixes this problem.

Signed-off-by: Khoa Huynh <khoa@us.ibm.com>

Add and use XendDomainInfo.getBackendFlags to decouple image.py from the
internals of XendDomainInfo, in preparation for changing the latter.

Signed-off-by: Ewan Mellor <ewan@xensource.com>

Changed dangerous default parameter values where used to use None instead.
Renamed variables type and ord since these clash with built-in names.  Renamed
unused variables to mark them as such.  Changed scheduler.py to use *args and
**kwargs rather than the strange non-idiomatic imitation of the same.

Signed-off-by: Ewan Mellor <ewan@xensource.com>

Remove the complexity of the config_handlers mechanism in favour of a simple
configure_maxmem method. The config_handlers mechanism was trying to be a
general configuration-registration framework, but that functionality was
unused and confusing.

Signed-off-by: Ewan Mellor <ewan@xensource.com>

Added allocateDeviceID, which uses the store to keep track of per-domain,
per-device-class device IDs on behalf of the DevController subclasses.

Signed-off-by: Ewan Mellor <ewan@xensource.com>

Move definition of xc and xend to avoid confusing pylint.

Signed-off-by: Ewan Mellor <ewan@xensource.com>

Renamed XendDomainInfo.getTargetMemory to getMemoryTarget, for consistency with
XendDomainInfo.setMemoryTarget.

Signed-off-by: Ewan Mellor <ewan@xensource.com>

Replaced the device handling mechanism used by XendDomainInfo.  Superficially,
this looks like the resurrection of DevController from controller.py, but the
mechanism is actually very different.  Device handling is now stateless inside
xend, relying on the store for state management, reducing DevController
instances to one-shot lifetimes.  Dev and its subclasses have gone completely.
The device creation code itself has moved from XendDomainInfo.createDevice into
DevController subclasses, and the previous contents of the subclasses has been
discarded (these subclasses were no longer being called, so all this code was
dead).

XendDomainInfo.getDeviceIds has gone, as it was unused.
XendDomainInfo.delete_device has been subsumed by XendDomainInfo.destroyDevice;
since device handling is now stateless inside xend, the distinction between
device 'deletion' and device 'destruction' is meaningless.

The s-expression describing devices has gone, as this information is no longer
available to xend in the same way, and seems to be unused.  If it is required,
it can be reinstated by loading device information from Xen or the store.

Signed-off-by: Ewan Mellor <ewan@xensource.com>

Add behaviour to the remove methods to remove the transaction's path itself.  This allows us to write Remove(path) to remove the specified path rather than having to slice the path ourselves.

In all cases, move the creation of a new transaction outside of the block
handling exceptions raised inside that transaction.  If the creation (start) of
the transaction fails, then t has not been assigned, and in any case no
transaction has been created, so it is wrong to attempt to abort that
(non-existent) transaction.

Signed-off-by: Ewan Mellor <ewan@xensource.com>

Trivial gdb build fix from Leendert.

Remove assignment to unused field memory_target.

Signed-off-by: Ewan Mellor <ewan@xensource.com>

Use XendDomainInfo.getDomain, getName to decouple XendDomain from the internals
of XendDomainInfo, prior to changing those internals.

Signed-off-by: Ewan Mellor <ewan@xensource.com>

Remove unused restoreFromDB, saveToDB.

Signed-off-by: Ewan Mellor <ewan@xensource.com>

Replace XendDomainInfo.setStoreChannel with XendDomainInfo.closeStoreChannel.
setStoreChannel was only ever called with channel=None (implying that the
current channel would be closed) so the other code there was superfluous.

Signed-off-by: Ewan Mellor<ewan@xensource.com>

Added getVCpuCount to XendDomainInfo and use that and getName, getDomain,
getTargetMemory inside image.py to decouple the latter from the internals of
the former.  Move the definition of xc inside image.py to please pylint.

Signed-off-by: Ewan Mellor<ewan@xensource.com>

Fix indentation.

Added getTargetMemory, getSsidref methods to XendDomainInfo, and use these and
getDomain, getName in SrvDomainDir, to decouple the latter from the internals of
the former, in preparation for changes internal to XendDomainInfo.

Signed-off-by: Ewan Mellor<ewan@xensource.com>

Issue warning if doxygen is not installed.

Signed-off-by: Ewan Mellor<ewan@xensource.com>

Changed ssidref default to 0 when parsing config file.

Signed-off-by: Ewan Mellor<ewan@xensource.com>

Added diagnostic message to exception when TypeError occurs on write.

Removed unused getDB method.

Signed-off-by: Ewan Mellor<ewan@xensource.com>

Moved xroot variable to not confuse pylint (and me).

Signed-off-by: Ewan Mellor<ewan@xensource.com>

Removed unused impotr and marked some parameters as unused.

Signed-off-by: Ewan Mellor<ewan@xensource.com>

Rename feilds to fields throughout.

Signed-off-by: Ewan Mellor<ewan@xensource.com>

Remove unused import, and unused class TwistedAdapter.  Mark one variable as
unused.

Signed-off-by: Ewan Mellor<ewan@xensource.com>

Added _1,.._5 to dummy-variables, so that we can specify multiple unused
parameters in our code.  Removed map and filter from bad-functions - I don't
think that we have any problem with functional programming styles!

Signed-off-by: Ewan Mellor<ewan@xensource.com>

Small cleanup to tpm split driver.

Signed-off-by: Stefan Berger <stefanb@us.ibm.com>

Remove native tpm drivers from the linux sparse tree and replace
with minimal patch in the automatically-applied patches/ directory.

Signed-off-by: Stefan Berger <stefanb@us.ibm.com>

Cleanup domain listing on xend start.  Fix reaper some more.
Signed-off-by: Christian Limpach <Christian.Limpach@cl.cam.ac.uk>

Add bindings for xs_get_domain_path().
Signed-off-by: Christian Limpach <Christian.Limpach@cl.cam.ac.uk>

Don't run reap() or domain_restarts() as side-effects of refresh.
Signed-off-by: Christian Limpach <Christian.Limpach@cl.cam.ac.uk>

Move dom0 setup code out of initial_refresh.
Signed-off-by: Christian Limpach <Christian.Limpach@cl.cam.ac.uk>

Remove pretty random cleanup code in _add_domain.
Signed-off-by: Christian Limpach <Christian.Limpach@cl.cam.ac.uk>